Skip to content

Add prefixing check#973

Merged
davidperezgar merged 8 commits into
trunkfrom
add-prefixing-check
Oct 19, 2025
Merged

Add prefixing check#973
davidperezgar merged 8 commits into
trunkfrom
add-prefixing-check

Conversation

@ernilambar
Copy link
Copy Markdown
Member

@ernilambar ernilambar commented Jun 26, 2025

Fixes #523

  • Added Prefixing_Check class
  • Scanner is added to find the potential prefixes in the plugin
  • Potential prefixes are used to utilize WordPress.NamingConventions.PrefixAllGlobals sniffs
  • Unit tests are added

@ernilambar ernilambar force-pushed the add-prefixing-check branch 17 times, most recently from 8842492 to 92519b6 Compare July 1, 2025 09:59
@ernilambar ernilambar force-pushed the add-prefixing-check branch from 92519b6 to 365424f Compare July 1, 2025 10:09
@ernilambar ernilambar marked this pull request as ready for review August 12, 2025 08:50
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 12, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: ernilambar <nilambar@git.wordpress.org>
Co-authored-by: frantorres <frantorres@git.wordpress.org>
Co-authored-by: davidperezgar <davidperez@git.wordpress.org>
Co-authored-by: swissspidy <swissspidy@git.wordpress.org>
Co-authored-by: jrfnl <jrf@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@davidperezgar davidperezgar added this to the 1.7.0 milestone Sep 23, 2025
Comment thread includes/Scanner/Prefix_Scanner.php Outdated
@davidperezgar davidperezgar merged commit c37b4dd into trunk Oct 19, 2025
26 checks passed
@davidperezgar davidperezgar deleted the add-prefixing-check branch October 19, 2025 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check: Generic function/class/define/option prefix names

3 participants